Skip to content

Added import.ps1#28

Open
rscholtelubberink wants to merge 5 commits into
mainfrom
added-import-functionality
Open

Added import.ps1#28
rscholtelubberink wants to merge 5 commits into
mainfrom
added-import-functionality

Conversation

@rscholtelubberink

@rscholtelubberink rscholtelubberink commented Jul 16, 2025

Copy link
Copy Markdown
Contributor

First version importscripts

@JeroenBL JeroenBL added the needs-review Indicates that the PR is ready and waiting for review label Jul 18, 2025
@rscholtelubberink rscholtelubberink added the current-sprint PRs to be addressed within the current sprint label Jul 25, 2025
Rick-Jongbloed
Rick-Jongbloed requested changes Aug 11, 2025
View reviewed changes

@Rick-Jongbloed Rick-Jongbloed left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed, please update the mapping logic so that Insite and Outsite are correctly mapped to their corresponding fields. This may also apply to additional fields if relevant.

@rhouthuijzen rhouthuijzen removed the current-sprint PRs to be addressed within the current sprint label Sep 26, 2025
@rhouthuijzen

rhouthuijzen commented Sep 26, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

On-Hold

@JeroenBL

JeroenBL commented Oct 7, 2025

Copy link
Copy Markdown

In de huidige implementatie worden gebruikers eerst in meerdere pagina’s opgehaald met een do/while-lus en vervolgens in een aparte foreach weggeschreven naar HelloID. Dit resulteert in dubbele iteraties en onnodig geheugenverbruik.

Het doel is om beide stappen te combineren: tijdens het ophalen van elke pagina direct de mapping uitvoeren en het resultaat outputten, zonder tussenopslag in $importedAccounts.

Huidige situatie

  • do/while-lus haalt gebruikerspagina’s op en voegt deze toe aan $importedAccounts.
  • foreach-lus doorloopt daarna alle $importedAccounts en schrijft resultaat naar HelloID

Gewenste situatie

  • Tijdens het ophalen van elke pagina (binnen de do/while-lus) direct en terugschrijven naar HelloID.
  • $importedAccounts vervalt volledig.

Voordelen

  • Minder geheugenverbruik
  • Minder code en eenvoudiger leesbaarheid
  • Betere performance bij grote datasets

Actiepunten

  1. Verwijder $importedAccounts-variabele
  2. Verplaats mapping-logica binnen de do/while-lus
  3. Test of alle records correct worden verwerkt bij meerdere pagina’s

@rscholtelubberink
Updated Import.ps1
1680b75 
Added insite/outsite
Removed importedAccounts variable and used direct streaming
@rscholtelubberink
Update import.ps1
3653b5b 
Fixes after testing
@rhouthuijzen rhouthuijzen added the on-hold Work is temporarily paused, pending additional information, decisions, or capacity. label Jan 30, 2026
@mverbraak-t4e mverbraak-t4e linked an issue Jul 3, 2026 that may be closed by this pull request
AcocuntReference of import differs from accountreference used in user lifecycle scripts #30
Open
@mverbraak-t4e

mverbraak-t4e commented Jul 3, 2026

Copy link
Copy Markdown

In de huidige implementatie worden gebruikers eerst in meerdere pagina’s opgehaald met een do/while-lus en vervolgens in een aparte foreach weggeschreven naar HelloID. Dit resulteert in dubbele iteraties en onnodig geheugenverbruik.

Het doel is om beide stappen te combineren: tijdens het ophalen van elke pagina direct de mapping uitvoeren en het resultaat outputten, zonder tussenopslag in $importedAccounts.

Huidige situatie

  • do/while-lus haalt gebruikerspagina’s op en voegt deze toe aan $importedAccounts.
  • foreach-lus doorloopt daarna alle $importedAccounts en schrijft resultaat naar HelloID

Gewenste situatie

  • Tijdens het ophalen van elke pagina (binnen de do/while-lus) direct en terugschrijven naar HelloID.
  • $importedAccounts vervalt volledig.

Voordelen

  • Minder geheugenverbruik
  • Minder code en eenvoudiger leesbaarheid
  • Betere performance bij grote datasets

Actiepunten

  1. Verwijder $importedAccounts-variabele
  2. Verplaats mapping-logica binnen de do/while-lus
  3. Test of alle records correct worden verwerkt bij meerdere pagina’s

@JeroenBL dit gebeurt hier toch niet? Ik zit hier met @rschouten97 en we begrijpen de comment niet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Rick-Jongbloed Rick-Jongbloed Awaiting requested review from Rick-Jongbloed

Requested changes must be addressed to merge this pull request.

Assignees

@rscholtelubberink rscholtelubberink

Labels

needs-review Indicates that the PR is ready and waiting for review on-hold Work is temporarily paused, pending additional information, decisions, or capacity.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AcocuntReference of import differs from accountreference used in user lifecycle scripts

5 participants

@rscholtelubberink @rhouthuijzen @JeroenBL @mverbraak-t4e @Rick-Jongbloed